Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add debug-level notification when package/test cannot be identified from test output #43

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

fredrikaverpil
Copy link
Owner

@fredrikaverpil fredrikaverpil commented Jun 24, 2024

This notification can be rather chatty in certain projects, so let's add a new dev-type option which can be used to enable it. The default is to have this disabled, as it will show in the test summary window that the test(s) were skipped.

The purpose of this notification is to try and catch bugs for when comparing a package/test from the go test test output with Neotest's position id.

@fredrikaverpil fredrikaverpil self-assigned this Jun 24, 2024
@fredrikaverpil fredrikaverpil changed the title Warn when test cannot be mapped between 'go test' output and Neotest position id Add warning option for test name from 'go test' output and Neotest position id mapping Jun 24, 2024
@fredrikaverpil fredrikaverpil force-pushed the add-warn-option branch 8 times, most recently from 5109468 to 956c2b4 Compare June 24, 2024 16:14
@fredrikaverpil fredrikaverpil marked this pull request as ready for review June 24, 2024 16:14
@fredrikaverpil fredrikaverpil changed the title Add warning option for test name from 'go test' output and Neotest position id mapping Add debug-level notification when package/test cannot be identified from test output Jun 24, 2024
@fredrikaverpil fredrikaverpil force-pushed the add-warn-option branch 4 times, most recently from 57f964a to ccf1637 Compare June 24, 2024 16:23
@fredrikaverpil fredrikaverpil enabled auto-merge (rebase) June 24, 2024 16:23
@fredrikaverpil fredrikaverpil merged commit ffba575 into main Jun 24, 2024
7 checks passed
@fredrikaverpil fredrikaverpil deleted the add-warn-option branch June 24, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant